Skip to content

Enhance Scrivito Template to resemble CoderDojo Website #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 8 commits into from
May 10, 2016

Conversation

nanophate
Copy link
Member

@nanophate nanophate commented May 6, 2016

やりたいこと

ScrivitoでCoderDojoみたいなレイアウトに変更する
補足 1 : Scrivito のレイアウト変更や追加の際はGemfileを利用する。
補足 2 : 自作のレイアウトや機能の場合はそのままプログラムに組み込める (release推奨)
Ref 1 : CMS Objects, Widgets, and Classes
Ref 2 : Scrivito Gem made by kostia
Ref 3 : Gemfile Best Practices & Discourse
Ref 4 : Cosmetic cleanup of generated Gemfile

やること

  • Gemfileの整理
    • Scrivitoの使用でレイアウトを追加&変更する際はGemを追加したり消したりする。
    • もし、様々なレイアウト機能や書籍に関するレイアウトが実装された場合追加されたGemの管理が重要になる
    • 調べてみたら、Gemfileの整理はちょっと難しいみたい 
    • How to find unused gems in my Gemfile
  • レイアウト変更に必要なCSSを追加する
    • 既存のファイルやrailsにコンフリクトをかけないようにする、念のためにドキュメントを書く、しっかりとプリリクに何を変更したかドキュメントする
    • すでに読み込まれてるCSSと読み込まれてないCSSを把握する。
    • 変更点を追加しウェブサイトのレイアウトを適用させる。

@nanophate nanophate added the 急ぎじゃないよ Make something better but not rushed. label May 6, 2016
@yasulab
Copy link
Member

yasulab commented May 7, 2016

👀 (now reading your code/description ...)

@yasulab
Copy link
Member

yasulab commented May 7, 2016

ん?このコミットは空コミットかな...? (Commit Messageと実態が違うような...)

936023a

@nanophate
Copy link
Member Author

多分、変更した内容をコピペしようと思った時に消してしまったんだと思います。
何をコミットしたかったのか分かるので、その変更点を後で書き足します。

@yasulab
Copy link
Member

yasulab commented May 8, 2016

何をコミットしたかったのか分かるので、その変更点を後で書き足します。

👍

@nanophate
Copy link
Member Author

nanophate commented May 10, 2016

外部の人がウェブサイトを変更するために必要なウェジェットを作る必要が出てきました。
(問い合わせ中)

利用するウェウジェトのリスト ( 強調している部分は自作する必要があります)

  • Title & Logo Section >ロゴとタイトル
  • 既存のウェジェットImage を利用  > CoderDojoの写真
    -** Centered Body Section (を使うとズレがひどい) > CoderDojo は7〜17歳...**
  • 既存のウェジェットQuote Layout を利用 > 道場に参加してみたい!...
  • Centered Grid Layout > 全国の道場...
  • 既存のウェジェットQuote Layout を利用 > CoderDojo をもっと知りたい...
  • Centered Footer > CoderDojo Japan 運営グループ....

screen shot 2016-05-10 at 7 14 35 pm

@nanophate nanophate closed this May 10, 2016
@nanophate nanophate reopened this May 10, 2016
@nanophate nanophate merged commit dfc5e07 into master May 10, 2016
@nanophate
Copy link
Member Author

nanophate commented May 10, 2016

間違ってClosedしてしまいました。😭元に戻せなくて泣きそうになりましたw

このPRを作った理由はScrivitoでCoderDojoみたいなレイアウトに変更することでしたが、新たにウェジェットを作る必要が出たためクローズします。

以下は三つ別々のPRを出します。

#3 Title & Logo Section >ロゴとタイトル
#9 Centered Grid Layout > 全国の道場...
#7 Centered Footer > CoderDojo Japan 運営グループ....

@yasulab
Copy link
Member

yasulab commented May 10, 2016

元に戻せなくて泣きそうになりましたw

大丈夫だから落ち着いてw

以下は三つ別々のPRを出します。

PRを細かく分けるのはいいね! 👍

@nanophate nanophate changed the title [WIP] Enhance Scrivito Template to resemble CoderDojo Website Enhance Scrivito Template to resemble CoderDojo Website May 10, 2016
@nanophate nanophate deleted the long-form-style-template branch May 11, 2016 07:58
@yasulab
Copy link
Member

yasulab commented May 13, 2016

Title & Logo Section >ロゴとタイトル
Centered Grid Layout > 全国の道場...
Centered Footer > CoderDojo Japan 運営グループ....

まずは、それぞれをIssue or PRとしてまずは書き出しましょう。
その後、それぞれにリンク貼った方が経緯がわかりやすいです。

次のような感じで、リンクを貼ると GitHub 上で相互に辿れるようになるので:
例: #3

yasulab pushed a commit that referenced this pull request Feb 17, 2018
最新の更新をマージ
yasulab pushed a commit that referenced this pull request Jul 4, 2023
Update _dojo.html+smartphone.haml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
急ぎじゃないよ Make something better but not rushed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants